Skip to content

fix typo in test runner link #2193

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 2 commits into from
Mar 9, 2025
Merged

fix typo in test runner link #2193

merged 2 commits into from
Mar 9, 2025

Conversation

@franciscotrenco
Copy link
Contributor

@franciscotrenco franciscotrenco commented Feb 25, 2025 •
edited by qodo-merge-pro bot
Loading

User description

There was a typo in the link to the Test Runner section.

Description

The link is found in the Documentation > WebDriver > Getting Started > Using Selenium page, under the 'Common Uses' headline and 'Testing' subheadline.

Motivation and Context

Currently the link does not go to anywhere.

Types of changes

  • Change to the site (I have double-checked the Netlify deployment, and my changes look good)
  • Code example added (and I also added the example to all translated languages)
  • Improved translation
  • Added new translation (and I also added a notice to each document missing translation)

Checklist

  • I have read the contributing document.
  • I have used hugo to render the site/docs locally and I am sure it works.

PR Type

Documentation

Description

  • Fixed a typo in the Test Runner link.

  • Updated the anchor text to correctly reference #test-runner.

Changes walkthrough 📝

Relevant files
Documentation
using_selenium.en.md
Fixed typo in Test Runner anchor link                                       

website_and_docs/content/documentation/webdriver/getting_started/using_selenium.en.md

  • Corrected the anchor link from #test-runners to #test-runner.
  • Ensured the link points to the correct section.
    		• +1/-1     
    Need help?
  • Type /help how to ... in the comments thread for any questions about Qodo Merge usage.
  • Check out the documentation for more information.
    •

    @franciscotrenco
    fix typo in test runner link
    d6ed938 
    There was a typo in the link to the Test Runner section in the
Documentation > WebDriver > Getting Started > Using Selenium page.
The link is found under the 'Common Uses' headline and 'Testing'
subheadline.
    @netlify
    Copy link

    netlify bot commented Feb 25, 2025 •
    edited
    Loading

    👷 Deploy request for selenium-dev pending review.

    Visit the deploys page to approve it

    Name Link
    🔨 Latest commit c307734

    @CLAassistant
    Copy link

    CLAassistant commented Feb 25, 2025 •
    edited
    Loading

    CLA assistant check
    All committers have signed the CLA.

    @qodo-merge-pro
    Copy link
    Contributor

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ No major issues detected

    @qodo-merge-pro
    Copy link
    Contributor

    qodo-merge-pro bot commented Feb 25, 2025 •
    edited
    Loading

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    harsha509
    harsha509 approved these changes Mar 9, 2025
    View reviewed changes
    Copy link
    Member

    @harsha509 harsha509 left a comment

    Choose a reason for hiding this comment

    The reason will be displayed to describe this comment to others. Learn more.

    Good catch 🚀

    Thank you @franciscotrenco !

    @harsha509
    Copy link
    Member

    Congratulations on your first contribution to Selenium Docs @franciscotrenco 🎉 🎉 🎉

    @harsha509 harsha509 merged commit 110dc1f into SeleniumHQ:trunk Mar 9, 2025
    3 checks passed
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Reviewers

    @harsha509 harsha509 harsha509 approved these changes

    Assignees

    No one assigned

    Labels

    Review effort 1/5

    Projects

    None yet

    Milestone

    No milestone

    Development

    Successfully merging this pull request may close these issues.

    3 participants

    @franciscotrenco @CLAassistant @harsha509